Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code reorganization and doc improvements #53

Closed
wants to merge 2 commits into from

Conversation

lann
Copy link
Collaborator

@lann lann commented Jun 21, 2024

Slicing this out of some ongoing work since it seems independently useful/reviewable:

  • Move ContentDigest to wasm-pkg-common
  • Add a bunch of doc comments, including a loader example
  • Reorganize loader "sources"
    • expose backend registry config structs

lann added 2 commits June 21, 2024 11:00
- "Sources" are currently just an internal implementation detail, so
  remove the public `source` module and move individual backend modules
  to the crate root.

- Rename Oci/WargConfig to Oci/WargRegistryConfig to make the
  relationship with RegistryConfig more clear.

- Add various documentation, including a wasm-pkg-loader example.
@lann
Copy link
Collaborator Author

lann commented Jun 21, 2024

In light of #54 I'm going to close this in favor of a PR implementing that rename.

@lann lann closed this Jun 21, 2024
@lann lann deleted the various-reorg branch June 21, 2024 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant